Skip to content

feat(helm): Add support for registry-specific values#701

Merged
Techassi merged 3 commits intomainfrom
template_a39137c
Apr 24, 2026
Merged

feat(helm): Add support for registry-specific values#701
Techassi merged 3 commits intomainfrom
template_a39137c

Conversation

@stackable-bot
Copy link
Copy Markdown
Contributor

Note

Autogenerated PR to update templated files since the last template run up to stackabletech/operator-templating@a39137c.

Trigger-message
@Techassi: Part of stackabletech/issues#716

Tip

Please update the change type as appropriate.

For example, if the change is breaking, add an ! after the change type:

chore!: Update templated files (b04cbc1)

If you need to add subsequent commits to make the checks pass, change the type:

refactor: Replace lazy_static with std::sync::LazyLock (b04cbc1)

For more information, See Conventional Commits.

@stackable-bot stackable-bot requested a review from Techassi April 24, 2026 06:53
@Techassi Techassi changed the title ci: Fix Helm Chart values file merge order feat(helm): Add support for registry-specific values Apr 24, 2026
NickLarsenNZ
NickLarsenNZ previously approved these changes Apr 24, 2026
Copy link
Copy Markdown
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread deploy/helm/secret-operator/templates/_local_helpers.tpl
@Techassi
Copy link
Copy Markdown
Member

I confirmed that make run-dev and make helm-install works as expected.

I also inspected and installed the packaged Helm Chart to ensure it looks and installs like we expect:

helm pull oci://oci.stackable.tech/sdp-charts/secret-operator:0.0.0-pr701 --untardir showmeboi --untar
helm install secret-operator oci://oci.stackable.tech/sdp-charts/secret-operator:0.0.0-pr701

Copy link
Copy Markdown
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Techassi Techassi added this pull request to the merge queue Apr 24, 2026
Merged via the queue into main with commit d46043b Apr 24, 2026
12 checks passed
@Techassi Techassi deleted the template_a39137c branch April 24, 2026 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants